-
Notifications
You must be signed in to change notification settings - Fork 341
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update v1alpha1 RayCluster CRD and controllers #22
Conversation
Bump the dependency versions - Golang 1.15 - Kubernetes 1.19.14 - controller-runtime 0.7.2 - controller-tools 0.5.0 Signed-off-by: Jiaxin Shan <seedjeffwan@gmail.com>
- Rename `WorkerGroupsSpec` to `WorkerGroupSpecs` - Remove HeadService and add ServiceType and EnableIngressin headGroupSpec - Add EnableInTreeAutoscaling field - Add rich information in RayClusterStatus Signed-off-by: Jiaxin Shan <seedjeffwan@gmail.com>
1. Refactor some internal methods to make invoking chains clear 2. Clean up some stale codes Signed-off-by: Jiaxin Shan <seedjeffwan@gmail.com>
Signed-off-by: Jiaxin Shan <seedjeffwan@gmail.com>
I am running and adding more tests to make sure it has good coverage. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
I know this PR is an exception, since it is the first big change. In the future we can try to keep the PRs smaller and more granular.
8ed8290
to
b3665af
Compare
This PR doesn't include ingress & in-tree autoscaler support in controllers. A separate PR will be cut later. After this change, future PRs will be small and manageable. |
* Bump k8s version to 1.19.x and golang to 1.15 Bump the dependency versions - Golang 1.15 - Kubernetes 1.19.14 - controller-runtime 0.7.2 - controller-tools 0.5.0 Signed-off-by: Jiaxin Shan <seedjeffwan@gmail.com> * Update v1alpha1 RayCluster CRD - Rename `WorkerGroupsSpec` to `WorkerGroupSpecs` - Remove HeadService and add ServiceType and EnableIngressin headGroupSpec - Add EnableInTreeAutoscaling field - Add rich information in RayClusterStatus Signed-off-by: Jiaxin Shan <seedjeffwan@gmail.com> * Update raycluster controller 1. Refactor some internal methods to make invoking chains clear 2. Clean up some stale codes Signed-off-by: Jiaxin Shan <seedjeffwan@gmail.com> * Update deploy manifests and examples Signed-off-by: Jiaxin Shan <seedjeffwan@gmail.com> * Address review comments
Address #13 #17 #21
/cc @akanso @chenk008 @chaomengyuan